-
Notifications
You must be signed in to change notification settings - Fork 10.4k
[IMP] Payroll: Updating contract templates #14807
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Hi @Felicious - this is ready for a review (peer and final). Thank you! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done with the content review, @larm-odoo ! I just had some minor rewording suggestions as comments. After you look those over, can you tag me again for a quick technical review? It'll be a quick one where I double-check for any typos and RST things and then we can merge! (:
41cf493
to
2f86873
Compare
Hi @Felicious - this is ready for another look =) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work with this improvement PR, @larm-odoo !
Just found really hard-to-spot typos and this is ready to go afterwards!
@robodoo delegate+
2f86873
to
8dea011
Compare
@robodoo r+ |
closes #14807 Signed-off-by: Lara Martini (larm) <larm@odoo.com>
Moving the create contract templates section from the main Payroll doc to this one.
Original task card for this PR.